Mcp core init#19
Merged
Merged
Conversation
- Fixed enum conflicts by standardizing RepositorySizeCategory usage - Updated all model configuration interfaces to include required properties - Fixed type compatibility issues between RepositoryProvider and DeepWikiProvider - Added proper imports and type assertions across core packages - Updated API layer to use proper enum types instead of string literals - Completed MCP hybrid package with comprehensive tool integration - Added maintenance and monitoring configurations - Added research documentation for MCP tool discovery - Updated session summaries and documentation All TypeScript builds now pass successfully without errors. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
alpsla
pushed a commit
that referenced
this pull request
Nov 15, 2025
…ixes) Fixed critical V9 report footer issues: 1. LSP Overlapping Ranges (lsp-sarif-converter.ts, lines 311-358) - Implemented proper range overlap detection algorithm - Prevents IDE corruption from conflicting edits - Algorithm: !(newEnd <= existingStart || newStart >= existingEnd) 2. Tool Performance Ranking Removed (metadata-footer.ts, lines 229-235) - Removed duplicate section showing hardcoded Java tools - Eliminates language-specific contamination - Added BUG FIX #19 comment 3. PR Comment Template Added (metadata-footer.ts, lines 348-356, 562-569) - Added actual ready-to-paste markdown template - Replaces empty 'Copy the markdown above' tip - Includes issue counts, severity stats, analysis time - Added BUG FIX #20 comment 4. Section Ordering Fixed (metadata-footer.ts) - Reorganized: How to Apply Fixes → PR Comment Template → Additional Files - Prioritizes actionable content before supplementary files Testing: - ✅ Verified all fixes in Spring PetClinic PR #950 (Java) - ✅ Tool Performance Ranking: Not found - ✅ PR Comment Template: Line 236 with actual markdown - ✅ LSP overlap detection: Algorithm implemented - ✅ Section ordering: Correct priority Files Modified: - src/two-branch/analyzers/lsp-sarif-converter.ts (47 lines) - src/two-branch/report/metadata-footer.ts (25 lines) Impact: Cleaner reports, no IDE corruption, better UX 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.